Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

replace to v0.9.40 for the farming feature ONLY #3

Merged
merged 3 commits into from
May 23, 2023

Conversation

b-yap
Copy link

@b-yap b-yap commented May 22, 2023

partially closes pendulum issue 219
update 3 packages:

  • bifrost-farming-rpc-runtime-api
  • bifrost-farming-rpc-api
  • bifrost-farming

@b-yap b-yap requested a review from a team May 22, 2023 08:08
Copy link
Member

@ebma ebma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume the compilation of the other pallets would not work anymore since you only added code changes to the farming pallet while updating the dependencies everywhere. Which is not problematic since we are only interested in the farming pallet anyways.
Did you test that the compilation of that pallet still works though? I can see that it relies on the local code in node/primitives. Just want to make sure that this does not require changes too.

It's a great idea to only add changes for the farming pallet and ignore the rest if possible IMO 👍

@b-yap
Copy link
Author

b-yap commented May 22, 2023

@ebma i ran cargo b and cargo test in all 3 modules

Copy link
Member

@ebma ebma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. LGTM then

@b-yap b-yap merged commit 5041a73 into polkadot-v0.9.40-farming May 23, 2023
@b-yap b-yap deleted the 219-update-runtimes-to-polkadot-v0940 branch May 23, 2023 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants